-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Removing git fetch --all #2083
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Removing git fetch --all #2083
Conversation
Do you use |
No absolutely not! Some might claim that it helps keeping things updated if you're working on your own fork and an upstream repo. As mentioned in the PR a colleague pointed out that it has the wrong description so I suggested a change. Taking the entry out entirely might however be a good option as well. |
Let's just remove it then; it's easy to put back if we ever decide it's important. |
Hi, A most common mistake is to not fetch correctly moved tags, and not to prune remotely removed branches and tags,
|
How else do you contribute to any project on GitHub that is not yours? You first fork, then push to your fork, then open a Pull Request in the forked repository, right? That's already two remotes. Sounds pretty common to me... |
So you'd prefer to keep the entry with the updated explanation? (that was the original title of this PR since |
Given that I, personally, rarely use I want it to be dropped for a valid reason, though: Because it is rarely useful. Not because users rarely have more than a single remote. |
Sure, Ill open a separate issue/PR. |
As almost everything in the Git project, this is a bit fuzzy. There is no official rule. It just so happens that @jvns, @To1ne and myself seem to be the only ones among those with the "Merging Powers" who care enough about this site to be the ones who end up commenting, assisting with, closing or merging Pull Requests. |
@dscho as an aside I don’t believe I have merging powers, could you merge this if you think it’s ready? |
That's definitely my mistake, which I hope to have rectified successfully.
Of course! |
@dscho (just for the record, because this discussion is long over) Yes, you have multiple remotes. But typically that's one remote for fetching/pulling and another for pushing.
In the Git project I have Junio ( Anyhow, no reason to keep it in the cheatsheet.
I think this should be considered to be put in the configuration (and for what it's worth I don't like I don't think we should put git config on the cheat sheet. But maybe we need a separate page for "recommended config"? cc @jvns |
Changes
git fetch --all
fetches all remotes. The cheat sheet stated that it's fetching all branches.Context
A colleague of mine pointed out, that
git fetch --all
doesn't fetch all branches but all remotes.git fetch --help
confirms that.After discussing in the PR I now suggest to remove it with an updated change.